Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move o.e.m2e.maven.runtime to m2e's git-repo root #1492

Merged
merged 2 commits into from
Jul 30, 2023

Conversation

HannesWell
Copy link
Contributor

Since o.e.m2e.maven.runtime is the only remaining module in m2e-maven-runtime there is no need anymore to keep m2e-maven-runtime as its parent project and the general build configuration separated.

@laeubi any objection?

@github-actions
Copy link

github-actions bot commented Jul 30, 2023

Test Results

106 files  ±0  106 suites  ±0   13m 13s ⏱️ + 3m 28s
660 tests ±0  650 ✔️ ±0  10 💤 ±0  0 ±0 
660 runs  ±0  649 ✔️ ±0  11 💤 ±0  0 ±0 

Results for commit 728d019. ± Comparison against base commit 69a6425.

♻️ This comment has been updated with latest results.

@HannesWell HannesWell force-pushed the topLevelMavenRuntime branch 3 times, most recently from 16071f1 to 8a2abfa Compare July 30, 2023 14:21
@HannesWell
Copy link
Contributor Author

Somehow the feature's build-qualifier does not take the version and qualifier change of o.e.m2e.maven.runtime into account:

[INFO] --- tycho-packaging:4.0.0:build-qualifier-aggregator (default-build-qualifier-aggregator) @ org.eclipse.m2e.feature ---
[INFO] The project's OSGi version is 2.4.100.20230705-1302

And consequently the feature is replaced with the baseline from the snapshot.

@laeubi
Copy link
Member

laeubi commented Jul 30, 2023

@laeubi any objection?

If you think it helps managing m2e build why not :-)

@laeubi
Copy link
Member

laeubi commented Jul 30, 2023

Somehow the feature's build-qualifier does not take the version and qualifier change of o.e.m2e.maven.runtime into account:

You need to bump the version manually, the qualifier update will likely miss non Tycho projects.

@HannesWell
Copy link
Contributor Author

Somehow the feature's build-qualifier does not take the version and qualifier change of o.e.m2e.maven.runtime into account:

You need to bump the version manually, the qualifier update will likely miss non Tycho projects.

Was this always required? IIRC in the past, this was not necessary. But this was likely in the time, when the m2e.maven.runtime participated as regular Eclipse Plugin in the build.

Do you think it would be feasible to also consider non Eclipse Plugin Projects when computing the qualifier?

@laeubi
Copy link
Member

laeubi commented Jul 30, 2023

The qualifier computation is quite fragile already and requires a special naming of artifacts/bundles so I'm not sure it is worth to invest more time as necessary but if you like you can debug the case and check why it is not considered.

Since o.e.m2e.maven.runtime is the only remaining module in
m2e-maven-runtime there is no need anymore to keep the general build
configuration separated.
and delete the now empty m2e-maven-runtime project.

Since o.e.m2e.maven.runtime is the only remaining module in
m2e-maven-runtime there is no need anymore to keep m2e-maven-runtime as
its parent project.
@HannesWell
Copy link
Contributor Author

The qualifier computation is quite fragile already and requires a special naming of artifacts/bundles so I'm not sure it is worth to invest more time as necessary but if you like you can debug the case and check why it is not considered.

Understand. Then lets keep this as task for the future. Since we will encounter this problem on every maven update, I think the chances are not too low, that one is annoyed enough bis this in the future.^^

@HannesWell HannesWell merged commit 0e7d484 into eclipse-m2e:master Jul 30, 2023
4 of 5 checks passed
@HannesWell HannesWell deleted the topLevelMavenRuntime branch July 30, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants